Skip to content

build(deps): bump github.com/josephburnett/jd/v2 to v2.4.0#1965

Merged
SamMorrowDrums merged 1 commit intogithub:mainfrom
alexandear-org:chore/upgrade-jd-v2
Feb 10, 2026
Merged

build(deps): bump github.com/josephburnett/jd/v2 to v2.4.0#1965
SamMorrowDrums merged 1 commit intogithub:mainfrom
alexandear-org:chore/upgrade-jd-v2

Conversation

@alexandear
Copy link
Contributor

@alexandear alexandear commented Feb 5, 2026 •
edited
Loading

Summary

Bumps github.com/josephburnett/jd/v2 to v2.4.0 in module dependencies.

Why

Because Dependabot can’t upgrade major versions, this bumps the dependency and drops a few indirect dependencies.

What changed

  • Updated github.com/josephburnett/jd/v2 to v2.4.0 in go.mod.
  • Refreshed go.sum for the dependency update.
  • Updated the third-party license files via runnning ./script/licenses.

MCP impact

  • No tool or API changes. Dependency-only update; no MCP surface changes.
  • Tool schema or behavior changed
  • New tool added

Prompts tested (tool changes only)

  • Not applicable (no tool changes).

Security / limits

  • No security or limits impact. Dependency version bump only.
  • Auth / permissions considered
  • Data exposure, filtering, or token/size limits considered

Tool renaming

  • I am renaming tools as part of this PR (e.g. a part of a consolidation effort)
    • I have added the new tool aliases in deprecated_tool_aliases.go
  • I am not renaming tools as part of this PR

Lint & tests

  • Linted locally with ./script/lint
  • Tested locally with ./script/test

Docs

  • Not needed
  • Updated (README / docs / examples)

@alexandear alexandear requested a review from a team as a code owner February 5, 2026 16:27
Copilot AI review requested due to automatic review settings February 5, 2026 16:27
Copilot AI reviewed Feb 5, 2026
View reviewed changes
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR upgrades github.com/josephburnett/jd from v1.9.2 to v2.4.0, bringing the go.mod dependency declaration in sync with the code which already imports the v2 module path. The upgrade also updates several indirect dependencies and removes some that are no longer needed.

Changes:

  • Upgraded main dependency github.com/josephburnett/jd from v1.9.2 to v2.4.0 (major version bump)
  • Updated indirect dependencies: go-openapi/jsonpointer, go-openapi/swag, and golang.org/x/exp
  • Removed unused indirect dependencies: rogpeppe/go-internal, yudai/golcs, gopkg.in/check.v1, and gopkg.in/yaml.v2

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.

File Description
go.mod Updates jd dependency from v1 to v2 and refreshes indirect dependencies
go.sum Updates checksums for all changed dependencies and removes entries for dropped dependencies

@SamMorrowDrums
Copy link
Collaborator

SamMorrowDrums commented Feb 9, 2026

@alexandear minor merge conflicts to deal with, but happy to merge this :-D thank you as always.

@alexandear
Copy link
Contributor Author

alexandear commented Feb 10, 2026

@alexandear minor merge conflicts to deal with, but happy to merge this :-D thank you as always.

@SamMorrowDrums Resolved conflicts, please review.

@SamMorrowDrums
Copy link
Collaborator

SamMorrowDrums commented Feb 10, 2026

Brilliant, thanks for the quick work.

I do also want to look at merging the linter change, I just wanted to have a moment to think about the eventual naming before merge!

@SamMorrowDrums SamMorrowDrums merged commit ccfec3d into github:main Feb 10, 2026
11 checks passed
@alexandear alexandear deleted the chore/upgrade-jd-v2 branch February 10, 2026 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@SamMorrowDrums SamMorrowDrums SamMorrowDrums approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@alexandear @SamMorrowDrums